Skip to content

Encode form name#1961

Merged
labkey-nicka merged 7 commits intodevelopfrom
fb_encodeFormName
Mar 30, 2026
Merged

Encode form name#1961
labkey-nicka merged 7 commits intodevelopfrom
fb_encodeFormName

Conversation

@labkey-nicka
Copy link
Copy Markdown
Contributor

@labkey-nicka labkey-nicka commented Mar 25, 2026

@labkey-nicka labkey-nicka merged commit 6ea7e0f into develop Mar 30, 2026
3 checks passed
@labkey-nicka labkey-nicka deleted the fb_encodeFormName branch March 30, 2026 22:06
labkey-matthewb added a commit to LabKey/platform that referenced this pull request Mar 30, 2026
#### Rationale
Whereas browsers seem to have trouble property formatting
multipart/form-data POSTs, we will encode form names that include these
three characters: `% " \`

For backward compatibility we will identify encoded field names by
prepending `%_`.

`quf_name` -> `quf_name`
`quf_why"?` -> `%_quf_why%22%3F`

#### Related Pull Requests
- LabKey/labkey-api-js#211
- LabKey/labkey-ui-components#1961
- LabKey/labkey-ui-premium#916
- LabKey/limsModules#2061
- #7515
- LabKey/testAutomation#2923

#### Changes
- <!-- list of descriptions of changes that are worth noting (replace
this comment) -->

---------

Co-authored-by: labkey-nicka <nickk@labkey.com>
labkey-matthewb added a commit to LabKey/testAutomation that referenced this pull request Mar 30, 2026
#### Rationale
Support an encoding scheme for INPUT.name to avoid problems with
multipart/form-data

#### Related Pull Requests
- LabKey/labkey-api-js#211
- LabKey/labkey-ui-components#1961
- LabKey/labkey-ui-premium#916
- LabKey/limsModules#2061
- LabKey/platform#7515
- #2923

#### Changes
- <!-- list of descriptions of changes that are worth noting (replace
this comment) -->

---------

Co-authored-by: labkey-nicka <nickk@labkey.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants